Skip to content

[Autocomplete] Fix TomSelect scroll back to top#3370

Open
MorcilloA wants to merge 1 commit intosymfony:2.xfrom
MorcilloA:fix/3369
Open

[Autocomplete] Fix TomSelect scroll back to top#3370
MorcilloA wants to merge 1 commit intosymfony:2.xfrom
MorcilloA:fix/3369

Conversation

@MorcilloA
Copy link
Copy Markdown

Q A
Bug fix? yes
New feature? no
Deprecations? no
Documentation? no
Issues Fix #3369
License MIT

Should fix the issue.

@carsonbot carsonbot added Autocomplete Bug Bug Fix Status: Needs Review Needs to be reviewed labels Mar 9, 2026
@carsonbot carsonbot changed the title fix: [Autocomplete] #3369 [Autocomplete] fix: #3369 Mar 9, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 9, 2026

📊 Packages dist files size difference

Thanks for the PR! Here is the difference in size of the packages dist files between the base branch and the PR.
Please review the changes and make sure they are expected.

FileBefore (Size / Gzip)After (Size / Gzip)
Autocomplete
controller.js 12.93 kB / 3.45 kB 13.07 kB+1% 📈 / 3.49 kB+1% 📈

@MorcilloA MorcilloA changed the title [Autocomplete] fix: #3369 [Autocomplete] Fix TomSelect scroll back to top Mar 9, 2026
@Kocal
Copy link
Copy Markdown
Member

Kocal commented Mar 9, 2026

Waiting for #3369 (comment)

@MatTheCat
Copy link
Copy Markdown
Contributor

Upstream issue is probably orchidjs/tom-select#556

I opened orchidjs/tom-select#821 at the time but it never made it.

@Kocal
Copy link
Copy Markdown
Member

Kocal commented Mar 19, 2026

Huh, that's a very bad news if the upstream repo is abandoned :s

@MatTheCat
Copy link
Copy Markdown
Contributor

That was more than one year ago; it seems the repository is active again.

@MorcilloA
Copy link
Copy Markdown
Author

Hello @MatTheCat ! Do you think it could be a good idea to reopen your PR there ?

@MatTheCat
Copy link
Copy Markdown
Contributor

I was expecting tons of conflicts but nothing 🤔 opened orchidjs/tom-select#1017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Autocomplete] Tom Select scrolls back to top if pagination is triggered with query term

5 participants